Skip to content

feat: per-author RSS feed, allow multiple article authors #351

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
sxd merged 10 commits into from
Aug 19, 2025
Merged

feat: per-author RSS feed, allow multiple article authors #351

sxd merged 10 commits into from
Aug 19, 2025

Conversation

@jsilvela
Copy link
Contributor

@jsilvela jsilvela commented Aug 9, 2025 •
edited
Loading

Takes advantages of Hugo's taxonomies to track article authors.

  • creates the new authors taxonomy
  • allows more than one author, updates the existing author rendering components
  • we get per-author RSS for free
  • modifies some of the CSS to display properly with more than one author.

Closes #133

@jsilvela jsilvela requested a review from a team as a code owner August 9, 2025 07:44
@jsilvela
Copy link
Contributor Author

jsilvela commented Aug 9, 2025

Some screen caps

Screenshot 2025-08-09 at 09-31-25 Screenshot 2025-08-09 at 09-10-14 Blog posts CloudNativePG - PostgreSQL Operator for Kubernetes

jsilvela
jsilvela commented Aug 9, 2025
View reviewed changes
layouts/authors/term.html
@@ -1,29 +1,27 @@
{{ define "main" }}
<h1 class="h1 text-center mx-auto md:w-3/5">{{.Title}}</h1>
<!-- <h1 class="h1 text-center mx-auto md:w-3/5">{{.Title}}</h1> -->
Copy link
Contributor Author

@jsilvela jsilvela Aug 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this h1 is currently broken in the production site, because the author pages had no title.
And, since we have the name again below for the Github link, not a big loss.

@jsilvela jsilvela changed the title feat: per-author RSS feed, allow multiple authors feat: per-author RSS feed, allow multiple article authors Aug 9, 2025
@jsilvela
Copy link
Contributor Author

jsilvela commented Aug 11, 2025

Vivaldi built-in RSS reader. Screengrabs.

2025-08-11 12 23 32 localhost e8071baece71 2025-08-11 12 24 43 localhost 8127f84caec4

@jsilvela jsilvela mentioned this pull request Aug 11, 2025
Merged
@jsilvela jsilvela force-pushed the dev/133 branch 2 times, most recently from 9c4dec3 to 36b6471 Compare August 13, 2025 07:08
@FloorD
Copy link
Contributor

FloorD commented Aug 14, 2025

I LOVE THIS

sxd
sxd reviewed Aug 14, 2025
View reviewed changes
@sxd
Copy link
Member

sxd commented Aug 14, 2025

@jsilvela I was testing and I found this:
Screenshot From 2025-08-14 18-34-50
There's some missing images to the authors in the latest post section, I'm checking if I can find the reason for it will keep you posted!

@sxd
Copy link
Member

sxd commented Aug 14, 2025

@jsilvela @FloorD I pushed a fix for the sidebar, just replicated what was been done before =) please review that I didn't screw up :D

@jsilvela
Copy link
Contributor Author

jsilvela commented Aug 15, 2025

Thanks @sxd ! I think it looks good at first glance. I forgot to test the sidebar, good catch.
Later today I hope to review your change.

@jsilvela
Copy link
Contributor Author

jsilvela commented Aug 15, 2025 •
edited
Loading

I added back the author name to the sidebar. Other than that your fix was good @sxd . Tested with multiple authors and it also works 👍

jsilvela and others added 10 commits August 19, 2025 10:37
@jsilvela @sxd
feat: new authors archetype, redo blog authorship
116baaf 
Signed-off-by: Jaime Silvela <[email protected]>
@jsilvela @sxd
chore: convert floor and jonathan to new format
97fb190 
Signed-off-by: Jaime Silvela <[email protected]>
@jsilvela @sxd
chore: move author index page to list mode
25055be 
Signed-off-by: Jaime Silvela <[email protected]>
@jsilvela @sxd
chore: finish refactoring
8f6834b 
Signed-off-by: Jaime Silvela <[email protected]>
@jsilvela @sxd
fix: restore authorship of bake blog post
27f3903 
Signed-off-by: Jaime Silvela <[email protected]>
@jsilvela @sxd
fix: markdownify footer content
ac5fa89 
Signed-off-by: Jaime Silvela <[email protected]>
@jsilvela @sxd
chore: delete useless field in archetype
54337a2 
Signed-off-by: Jaime Silvela <[email protected]>
@jsilvela @sxd
chore: convert new posts
15ed509 
Signed-off-by: Jaime Silvela <[email protected]>
@sxd
chore: improve sidebar images with the new list of authors
2f81bce 
Signed-off-by: Jonathan Gonzalez V. <[email protected]>
@jsilvela @sxd
chore: add back author name in sidebar
5018e95 
Signed-off-by: Jaime Silvela <[email protected]>
@sxd sxd merged commit 43f0cbc into main Aug 19, 2025
4 checks passed
@sxd sxd deleted the dev/133 branch August 19, 2025 08:41
github-actions bot pushed a commit that referenced this pull request Aug 19, 2025
@sxd
feat: per-author RSS feed, allow multiple article authors (#351)
51e9e33 
Signed-off-by: Jaime Silvela <[email protected]>
Signed-off-by: Jonathan Gonzalez V. <[email protected]>
Co-authored-by: Jonathan Gonzalez V. <[email protected]> 43f0cbc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sxd sxd sxd approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add per-author RSS feed, possibly with filters

3 participants

@jsilvela @FloorD @sxd